The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 4, 2026 11:07pm

PR Summary

Medium Risk
Upgrades posthog-js/posthog-node and adjusts SIGINT/SIGTERM shutdown behavior; the main risk is subtle runtime changes in analytics delivery/flushing and process shutdown ordering.

Overview
Upgrades PostHog SDK dependencies (posthog-js and posthog-node) and updates the lockfile to match.

Moves PostHog shutdown/flush handling out of OpenTelemetry initialization: OTel now only shuts down its own SDK, while register() always registers a separate SIGINT/SIGTERM handler to call getPostHogClient()?.shutdown() so events can flush even when OTel is disabled or not configured.

^{Reviewed by Cursor Bugbot for commit 6ab5fa5. Configure here.}

@greptile

@cursor review

Greptile Summary

This PR fixes a real bug where the PostHog shutdown() call was only registered when OpenTelemetry was also enabled. By moving the shutdownPostHog signal handler from inside initializeOpenTelemetry() to the top-level register() function, PostHog events will now be properly flushed on process exit regardless of telemetry configuration. The two PostHog SDK upgrades (posthog-js 1.334.1 → 1.364.4, posthog-node 5.9.2 → 5.28.9) are routine maintenance.

Key changes:

• shutdownPostHog handler moved from initializeOpenTelemetry() (which exits early if OTel is disabled) to register(), guaranteeing SIGTERM/SIGINT coverage on all deployments
• posthog-js bumped from 1.334.1 to 1.364.4
• posthog-node bumped from 5.9.2 to 5.28.9
• Lock file updated accordingly

Confidence Score: 5/5

Safe to merge — the change is a straightforward and correct refactor with no logic regressions.

The OTel/PostHog separation is exactly right: OTel shutdown belongs inside initializeOpenTelemetry() because it depends on the sdk instance, while PostHog shutdown belongs in register() since it must always run. No P0 or P1 issues found.

No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant Next as Next.js Runtime
    participant Reg as register()
    participant OTel as initializeOpenTelemetry()
    participant PH as PostHog Client
    participant OS as OS Signal

    Next->>Reg: call register()
    Reg->>OTel: await initializeOpenTelemetry()
    alt OTel enabled
        OTel->>OTel: start SDK
        OTel->>OS: register shutdownOtel on SIGTERM/SIGINT
    else OTel disabled
        OTel-->>Reg: return early (no OTel handlers)
    end
    Reg->>OS: register shutdownPostHog on SIGTERM/SIGINT
    Note over Reg,OS: PostHog handler always registered

    OS->>OTel: SIGTERM/SIGINT (if OTel enabled)
    OTel->>OTel: sdk.shutdown()
    OS->>PH: SIGTERM/SIGINT
    PH->>PH: flush & shutdown

_{Reviews (1): Last reviewed commit: "fix(posthog): revert flushAt to 20 for l..." | Re-trigger Greptile}